Skip to content

Getting started and content based on OR V1.6 added #42

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Getting started and content based on OR V1.6 added #42

wants to merge 3 commits into from

Conversation

sweiland-openrails
Copy link
Contributor

Getting started based on OR V1.6 added
Content changed

@sweiland-openrails sweiland-openrails marked this pull request as draft July 13, 2025 12:19
cjakeman
cjakeman reviewed Jul 17, 2025
View reviewed changes
Copy link
Contributor

@cjakeman cjakeman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Siebren,

Thanks for what you've done here.

As a native English speaker, I've made some corrections to the language.
I've also chosen to add the Download button and an image of Demo Model 1, as I think this will be friendlier than a wall of text.

I see what you've done by using an Explore first and then following that with the 09:30 activity. I will approve the change to openrails.content to support this.

It looks as though Github will let me Approve your PR plus my changes, which I find odd. I won't attempt that until after you've looked at what I've done and made a comment.

Best wishes,

Chris

@sweiland-openrails sweiland-openrails marked this pull request as ready for review July 17, 2025 17:53
@sweiland-openrails
Copy link
Contributor Author

hi Chris,

Your fixes are ok, thanks for looking into this. Adding the extra blanks lines makes it more readable.

Made one more fix: removed the Waverley picture from the "Getting Started" print.

regards, Siebren.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cjakeman cjakeman cjakeman left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sweiland-openrails @cjakeman